Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Sep 30, 2025

Part of our style guide is to use sentence case for headings. The Reference was very inconsistent with this. This updates all the headings to use sentence case.

@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Sep 30, 2025
@@ -1,5 +1,5 @@
r[abi]
# Application Binary Interface (ABI)
# Application binary interface (ABI)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

paused seeing this one, then "nah, it good" (then I checked how wikipedia does it, and it matches your change)

Copy link
Member

@joshtriplett joshtriplett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I read through the entire commit, and it looks reasonable. Good to have consistency here.

(Would love to see this automatically checked by CI, but that can come later.)

Part of our style guide is to use sentence case for headings. The
Reference was very inconsistent with this. This updates all the headings
to use sentence case.
@traviscross traviscross force-pushed the reference-sentence-case branch from 094ce41 to 98cce88 Compare October 7, 2025 21:30
@rustbot
Copy link
Collaborator

rustbot commented Oct 7, 2025

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@traviscross traviscross enabled auto-merge October 7, 2025 21:31
@traviscross traviscross added this pull request to the merge queue Oct 7, 2025
Merged via the queue into rust-lang:master with commit 792ec17 Oct 7, 2025
5 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants